-
Notifications
You must be signed in to change notification settings - Fork 129
Add readiness probe for NGINX on pod startup #3629
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
7cb82dc
to
1b718a5
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #3629 +/- ##
==========================================
+ Coverage 87.02% 87.10% +0.08%
==========================================
Files 127 127
Lines 15471 15517 +46
Branches 62 62
==========================================
+ Hits 13463 13516 +53
+ Misses 1853 1848 -5
+ Partials 155 153 -2 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
fad90d4
to
a48dbf0
Compare
572c5b4
to
5dd1fd8
Compare
5dd1fd8
to
19bb3ec
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice job! just a couple small things
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice job, lgtm after single comment and Saylor's comments
Proposed changes
Write a clear and concise description that helps reviewers understand the purpose and impact of your changes. Use the
following format:
Problem: Users want to have a readiness probe for NGINX on startup to verify pod health.
Solution: Expose readiness probe configuration fields in the
NginxProxy
spec, allowing users to customize them as needed. By default, deploy NGINX with a readiness probe at the/readyz
endpoint.Testing: Manual testing
values.yaml
Nginx proxy Config
Nginx starts successfully, Nginx Proxy config has correct values for readiness probe fields, Nginx deployment reflects the same
Nginx restarts a new pod successfully, Nginx deployment reflects the same, server block port is updated
DaemonSet
as well.Please focus on (optional): If you any specific areas where you would like reviewers to focus their attention or provide
specific feedback, add them here.
Closes #3594
Checklist
Before creating a PR, run through this checklist and mark each as complete.
Release notes
If this PR introduces a change that affects users and needs to be mentioned in the release notes,
please add a brief note that summarizes the change.